Skip to content

✅ Add Salesforce LWC Test Application. - #4757

Merged
BeltranBulbarellaDD merged 25 commits into
mainfrom
salesforce/add-e2e-tests_2
Jun 29, 2026
Merged

✅ Add Salesforce LWC Test Application.#4757
BeltranBulbarellaDD merged 25 commits into
mainfrom
salesforce/add-e2e-tests_2

Conversation

@BeltranBulbarellaDD

Copy link
Copy Markdown
Contributor

Motivation

To be able to e2e test Salesforce LWC, we need a minimal application we can deploy and test on.

Changes

Creates new test app under test/apps/sf-lwc-app which contains 2 screens and several testing buttons which allow to test the rum-slim bundle against salesforce envirnments.

It is not yet hooked up into the build-apps script so for now it contains inside the folder a script to upload the changes to the engrum Salesforce Sandbox which does not expire.

Test instructions

Run the sf-lwc-app deploy script with any changes on the rum-slim bundle. Then log into the deployed application here. (Use 1Password beltran.bulbarella@datadoghq.com.engrumdev user)

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.
  • Updated documentation and/or relevant AGENTS.md file

@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Jun 10, 2026

Copy link
Copy Markdown

Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 77.00% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 4175f81 | Docs | Datadog PR Page | Give us feedback!

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7da Bot commented Jun 10, 2026

Copy link
Copy Markdown

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 172.42 KiB 172.42 KiB 0 B 0.00%
Rum Profiler 8.22 KiB 8.22 KiB 0 B 0.00%
Rum Recorder 21.14 KiB 21.14 KiB 0 B 0.00%
Logs 54.37 KiB 54.37 KiB 0 B 0.00%
Rum Slim 129.95 KiB 129.95 KiB 0 B 0.00%
Worker 22.96 KiB 22.96 KiB 0 B 0.00%

@BeltranBulbarellaDD

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 33581fd399

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread test/apps/sf-lwc-app/scripts/setup.mjs
Comment thread test/apps/sf-lwc-app/force-app/main/default/lwc/datadogInit/datadogInit.js Outdated
Comment thread test/apps/sf-lwc-app/scripts/setup.mjs
@BeltranBulbarellaDD
BeltranBulbarellaDD marked this pull request as ready for review June 24, 2026 14:35
@BeltranBulbarellaDD
BeltranBulbarellaDD requested a review from a team as a code owner June 24, 2026 14:35

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0560c00549

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread test/apps/sf-lwc-app/force-app/main/default/lwc/datadogInit/datadogInit.js Outdated
Comment thread test/apps/sf-lwc-app/force-app/main/default/lwc/datadogInit/datadogInit.js Outdated
trackLongTasks: true,
trackResources: true,
trackUserInteractions: true,
beforeSend: (event) => {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❓ question:** ‏why do we need to clean?

@BeltranBulbarellaDD BeltranBulbarellaDD Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because we need to append the SDK config to the url, so the view name will be created with the config. So to clean a little bit. But it doesn't have to be this way. We can just not clean.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 07a08a229a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread test/apps/sf-lwc-app/force-app/main/default/lwc/datadogInit/datadogInit.js Outdated
@BeltranBulbarellaDD
BeltranBulbarellaDD merged commit 29e2920 into main Jun 29, 2026
31 checks passed
@BeltranBulbarellaDD
BeltranBulbarellaDD deleted the salesforce/add-e2e-tests_2 branch June 29, 2026 15:11
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants